Conversation
The Active Agents repo scan treated the opened workspace path as the repo root. In a second VS Code window opened on a linked worktree or subfolder, that hid the owning repo's active-session files. Resolve the owning repo root from git metadata before reading sessions, keep template parity, and cover the linked-worktree window with a regression test. Constraint: Second VS Code windows may open on linked worktrees or repo subfolders, not only the owning repo root Rejected: Aggregate every parent-workspace daemon session | would leak unrelated repo agents into a repo-scoped window Confidence: high Scope-risk: narrow Reversibility: clean Directive: Resolve workspace folders to owning repo roots before reading Active Agents session state Tested: NODE_PATH=/home/deadpool/Documents/recodee/gitguardex/node_modules node --test test/vscode-active-agents-session-state.test.js Tested: openspec validate agent-codex-show-active-agents-in-second-vscode-wind-2026-04-23-16-50 --type change --strict Tested: openspec validate --specs (no items found to validate) Not-tested: Manual two-window VS Code clickthrough
NagyVikt
added a commit
that referenced
this pull request
Apr 23, 2026
The feature lane merged cleanly, but the cleanup boxes in the change record stayed unchecked because the finish flow stopped at local branch deletion while the /tmp worktree was still attached. Record the real PR, merge commit, and prune evidence so the merged change remains truthful on main. Constraint: Cleanup evidence must match post-merge branch and worktree state, not pre-finish intent Rejected: Leave the merged tasks file with unchecked cleanup boxes | makes the OpenSpec closeout inaccurate Confidence: high Scope-risk: narrow Reversibility: clean Directive: If finish exits after merge, capture PR state with gh and then backfill tasks.md from a follow-up lane Tested: openspec validate agent-codex-show-active-agents-in-second-vscode-wind-2026-04-23-16-50 --type change --strict Not-tested: Additional automated suites; docs-only bookkeeping change Co-authored-by: NagyVikt <nagy.viktordp@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated by gx branch finish (PR flow).